Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code Scanning API #905

Merged
merged 6 commits into from
Oct 19, 2023
Merged

Code Scanning API #905

merged 6 commits into from
Oct 19, 2023

Conversation

bjoernricks
Copy link
Contributor

What

Extend and activate the GitHub Code Scanning API

Why

Add missing REST calls.

References

DEVOPS-817

Checklist

  • Tests

@bjoernricks bjoernricks requested a review from a team as a code owner October 12, 2023 13:52
@greenbonebot greenbonebot enabled auto-merge (squash) October 12, 2023 13:52
@github-actions
Copy link

github-actions bot commented Oct 12, 2023

Conventional Commits Report

Type Number
Added 5

🚀 Conventional commits found.

@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

Merging #905 (b4c64a3) into main (070f2de) will increase coverage by 0.06%.
The diff coverage is 93.00%.

@@            Coverage Diff             @@
##             main     #905      +/-   ##
==========================================
+ Coverage   90.06%   90.13%   +0.06%     
==========================================
  Files          90       90              
  Lines        6061     6203     +142     
  Branches      859      877      +18     
==========================================
+ Hits         5459     5591     +132     
- Misses        418      423       +5     
- Partials      184      189       +5     
Files Coverage Δ
pontos/github/models/code_scanning.py 100.00% <100.00%> (ø)
pontos/github/api/api.py 87.34% <75.00%> (-0.66%) ⬇️
pontos/github/api/code_scanning.py 88.54% <88.31%> (-0.55%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@bjoernricks bjoernricks marked this pull request as draft October 12, 2023 13:56
auto-merge was automatically disabled October 12, 2023 13:56

Pull request was converted to draft

@bjoernricks bjoernricks marked this pull request as ready for review October 17, 2023 14:47
@bjoernricks bjoernricks enabled auto-merge (rebase) October 17, 2023 14:47
Allow to use the GitHub code scanning API
Add GitHub API for getting and deleting code scanning analyses.
Add the GitHub API for their CodeQL based scanning.
Implement the code scanning default setup API of GitHub.
Implement the two missing APIs for the GitHub code scanning API handling
SARIF data.
It seems that gzip.compress creates different data with Python < 3.11
and therefore the sarif data passed to the http post can't be checked in
the tests.
@bjoernricks bjoernricks merged commit b57030d into main Oct 19, 2023
21 checks passed
@bjoernricks bjoernricks deleted the code-scanning-api branch October 19, 2023 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants